[ported from cavs2.5-001-drop-stable] ssp-related changes#9835
Merged
abonislawski merged 4 commits intothesofproject:jsl-005-drop-stablefrom Feb 17, 2025
Merged
[ported from cavs2.5-001-drop-stable] ssp-related changes#9835abonislawski merged 4 commits intothesofproject:jsl-005-drop-stablefrom
abonislawski merged 4 commits intothesofproject:jsl-005-drop-stablefrom
Conversation
If DMA is active, do not read data directly from the SSP RX fifo. Signed-off-by: Kai Vehmanen <[email protected]> (cherry picked from commit e94071f) (cherry picked from commit 736af01)
If DMA transaction is ongoing when RX is enabled, this can lead to stuck communication between DMA and SSP (DMA service request not seen by the DMA). To avoid this, flush the RX fifo before enabling SSP RX. Link: thesofproject#7548 Suggested-by: Peter Ujfalusi <[email protected]> Signed-off-by: Kai Vehmanen <[email protected]> (cherry picked from commit 4a4d8d2) (cherry picked from commit 2c3f370)
The receive FIFO needs to be drained in a different way depending when it is done. - before start If the RX FIFO is in overflow state then we must read all the entries out to empty it (it was after all full). - before stop The DMA might be already running to read out data. Check the FIFO level change in one sample time which gives us the needed information to decide to wait for another loop for the DMA burst to finish, wait for the DMA to start it's burst (DMA request was asserted) or drain the FIFO directly. No need to drain the RX fifo at probe time. Signed-off-by: Peter Ujfalusi <[email protected]> (cherry picked from commit 69d56a7) (cherry picked from commit 6571598)
Dynamic pipeline breaks mclk always-on feature by freeing dai component in HW_FREE stage; the ssp_remove() will be called and disables mclk unconditionally. Signed-off-by: Brent Lu <[email protected]> (cherry picked from commit 6dc474c) (cherry picked from commit d5e3c70)
lgirdwood
approved these changes
Feb 17, 2025
Member
lgirdwood
left a comment
There was a problem hiding this comment.
@abonislawski fyi for merge
abonislawski
approved these changes
Feb 17, 2025
Member
|
@abonislawski I assume you will merge for stable release branches ? |
Member
|
@lgirdwood yes, of course |
0560990
into
thesofproject:jsl-005-drop-stable
4 of 15 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Update some ssp-related commits from cavs2.5-001-drop-stable to jsl-005-drop-stable.
(we didn't do cherry-picking from main directly because the code has been diverted by the age. cavs2.5-001-drop-stable is the nearest branch and has been verified as good.